-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update README.md #1
Conversation
README.md
Outdated
@@ -25,7 +25,7 @@ jobs: | |||
build: | |||
runs-on: self-hosted | |||
steps: | |||
- uses: AutoModality/action-clean@v1 | |||
- uses: freenet-actions/action-clean@v1.1.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ich meine, dass die Konvention für GH-Actions und Versionsnummern ist, dass man bei der Anwendung nur die Major-Version angibt. In diesem Fall also @v1
.
v1
zeigt auf den gleichen Commit wie v1.1.2
.
Deswegen sollte das klappen.
Ich hab mich jetzt auch gefragt, warum wir überhaupt geforkt haben. Musste Hinnerk fragen.
1.1.1 hatte einen Bug, sodass nicht mehr korrekt gelöscht wurde.
Wenn man 1.1.0 verwendet hat, hat sich aber dependabot beschwert, dass es eine neuere Version gab.
Vielleicht sollten wir diesen Hinweis noch mit in die README aufnehmen.
Update: AutoModality#6 (comment) hier war unsere Diskussion dazu.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ohh, das mit der Konvention wusste ich nicht
Ich kann versuchen noch einen Hinweis in der README einzubauen
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Habe noch was zum Fork geschrieben, gerne nochmal gucken :)
Damit das Beispiel auch die Action aus diesem Repository verwendet